wayland: Fix toplevel lookup before starting DnD

We use the high-level gdk_device_get_window_at_position() to figure
out the window, although this one actually tries to find out the
current window under the device coordinates, which might well fall
outside the window, so NULL is returned in those cases.

Fix this by using the lower level _gdk_device_window_at_position()
that will return the toplevel without further lookups, so is more
desirable here.

https://bugzilla.gnome.org/show_bug.cgi?id=758250

gtkdnd: Use gdk_drag_begin_for_device()

We've got the pointer at hand there, just pass that instead of figuring
out from the client pointer.

wayland: Implement GdkDevice::window_at_position for touch

This goes through its own master pointer, so look up the pointer
emulating touch focus window and coordinates.

wayland: Disallow setting the cursor on the touch master pointer

That "pointer" is not backed by anything in the windowing system, ignore
cursor updates there.

wayland: Ensure we get the master pointer on GdkDeviceManager::get_client_pointer

Now that we have multiple master pointers, this call may pick the wrong one.
Instead, pick the GdkWaylandDeviceData from the first device, and pick the
master pointer from there.

wayland: Separate touch pointer emulation into its own master pointer

The common GDK code accounts for "pointer emulating" touch sequences to be
synchronized with the pointer position by the windowing system.

However on Wayland pointer and touch are completely independent, the backend
attempts to implement pointer emulation, but doesn't account for the
possible crossing events happening when the user switches from pointer to
touch or the opposite.

In order to fix this, and to ensure we don't have to interact with the
master pointer (which backs the wl_pointer), separate the touch interface
to have its own master pointer, and ensure crossing events are emitted on
it, so the picture of an "emulated pointer" is complete above the backend.

Inspired in a former patch by Jonny Lamb <jonnylamb@gnome.org>

https://bugzilla.gnome.org/show_bug.cgi?id=750845

gtkheaderbar: update the window buttons on ::hierarchy-changed instead of ::realize

The window button setup depends on properties of the toplevel window.
Instead of updating the setup on realize, do it when the toplevel
changes.

This makes sure that when a GtkHeaderBar is added to a window
all the widgets are present and get_preferred_height() will return
the height the widget will have when finally shown. This allows
the logic in gtkwindow to select the right window size so that
the content size will match the requested default size.

https://bugzilla.gnome.org/show_bug.cgi?id=756618

gtkwindow: apply CSD adjustments to the default size when used instead of when setting it

Before the resulting window size would differ if the default size was set
before adding a headerbar vs after. Now the saved state is again the actual
requested size and it is adjusted at the time we request a window size.

https://bugzilla.gnome.org/show_bug.cgi?id=756618

window: maximize on double click only if allowed

GtkHeaderBar will not show the maximize button if the window in not of
type normal or not resizeable.

Use the same restriction for double-click actions as well.

Bugzilla: https://bugzilla.gnome.org/show_bug.cgi?id=757530

Fix return annotations for GtkWindow

Add nullable annotations for functions that may return NULL.

https://bugzilla.gnome.org/show_bug.cgi?id=753520

Fix character entities

These are not supposed to be supported by gtk-doc¹

¹https://bugzilla.gnome.org/show_bug.cgi?id=758137

https://bugzilla.gnome.org/show_bug.cgi?id=758175

gtkstack: allow templates to override all properties

Currently GtkStack has some G_PARAM_CONSTRUCT properties. That means,
the properties are set with its default value after the initializacion
of the object.
When using GtkBuilder to build objects, GtkBuilder creates them and
after that sets the properties found on the xml definition.
However, this is not true for templates because the template is initialized
in the init() function of the actual object, and after that, the construct
properties will be set.

This is a problem when someone wants to use templates with GtkStack and
set those properties, since they will be set on the tempalt initialization
and set again to its default values afterwards.

To fix this, make those properties not G_PARAM_CONSTRUCT.

https://bugzilla.gnome.org/show_bug.cgi?id=758086

Updates

wayland: Avoid a crash

We can end up in situations where NULL gets passed to
get_transient_parent(). Embrace it instead of avoiding it.

https://bugzilla.gnome.org/show_bug.cgi?id=758327

Add more CSS node docs

Trivial: Some doc rewording

Add some CSS node docs

Write up some of the insights from doing the CSS node transition,
so the knowledge is not lost.

Do not use the GPL v3 blurb for LGPL v2.1 files

GTK+ is licensed under the terms of the GNU LGPL v2.1+.

Do not use the LGPL v3 blurb for LGPL v2.1 files

GTK+ is released under the terms of the GNU LGPL v2.1+.

Fix the license version in GtkTrashMonitor

There is no GNU Lesser General Public License version 2; it's either GNU
Library General Public License version 2, or GNU Lesser General Public
License version 2.1.

Fix license version for GtkFileChooserWidget private header

Since it's using Lesser GPL, use version 2.1; there is no Lesser GPL v2,
as it was called 'Library GPL' prior to v2.1.

Fix the licensing blurb of GtkBookmarksManager

Copy-pasta from GPL instead of LGPL.

Also, there is no GNU Lesser General Public License version 2; either
it's the GNU Library General Public License version 2, or it's the GNU
Lesser General Public License version 2.1.

Fix the licensing blurb in GtkPlacesSidebar

The code is released under the terms of the LGPL v2.1+, as is the rest
of the GTK+ code.

Fix license header

I didn't meant to make this different from any other file
in GTK+ when I added it.

box: Use an element name

toolitemgroup: Fix initial state

GtkToolItemGroup starts out as non-collapsed, make the CSS node
state reflect that.

Document GDK_TOUCHPAD_GESTURE_MASK

entry: ensure valid GdkWindow before set_invisible_cursor()

If the window has not yet been created, then we can't set the invisible
cursor yet. This can happen in situations where the widget is in a
revealer with type-to-search functionality.

HighContrast: remove deprecated progressbar style properties

These are ignored and trigger a runtime warning.

Updated Spanish translation

gtkwindow: apply csd offset to set/get_default_size

An application may use gtk_window_get_size() to retrieve the current
window size and later reuse that size with
gtk_window_set_default_size().

gtk_window_set_default_size() and gtk_window_get_default_size() should
also take client side decorations offset into account.

Bugzilla: https://bugzilla.gnome.org/show_bug.cgi?id=756618

popover: Avoid a crash when no relative-to widget is set

This was showing up in glade, which creates freestanding
popovers.

HighContrast: remove deprecated style properties

expander: Use the right state when querying style properties

Failure to do so causes warnings.

gtk-demo: Fix the CSS examples

The CSS needs small adjustments after all the CSS node changes.

gtk-demo: Fix the assistant example

GtkEntry respects valign now, so we need to set it to something
other than fill.

Adwaita: Drop deprecated style properties

wayland: Move additional pointer buttons after the old 4-7 scrolling ones

We were using that range for the extra buttons after left/right/middle,
while this is harmless for clients not handling extra buttons (we
used to translate those button events into scroll events in x11 anyway)
this will be unexpected for clients that do handle additional mouse
buttons themselves (eg. back/forward buttons present in some mice).

In order to remain compatible with X11, those need to be assigned from
button 8 onwards.

Also, include input.h, and stop using magic numbers here.

https://bugzilla.gnome.org/show_bug.cgi?id=758072

Updated Spanish translation

Updated Spanish translation

Updated Scottish Gaelic translation

x11: Add "pointer" to the is-not-a-touchscreen device name checks

Commit 1266d15c4 also broke Xwayland, as it does the same trick
than VMWare pointers. Let's extend the heuristic to check for "pointer"
in the device name, what can possibly go wrong...

https://bugzilla.gnome.org/show_bug.cgi?id=757358

stackswitcher: add GtkStackSwitcher:icon-size property

Set the GtkStackSwitcher:icon-size property to change the size of icons
used in the linked GtkRadioButton children.

https://bugzilla.gnome.org/show_bug.cgi?id=758208

x11: Try harder to find a core pointer

We currently just look for a master device with input source MOUSE.
After recent changes to the way input devices are classified, xwayland
on my system comes up with a virtual core pointer that has input
source TOUCHSCREEN. This was causing assertion failures. Be a little
more careful and accept a touchscreen as core pointer, if there is
no mouse.

Adwaita: Remove deprecated style properties

These are now triggering warnings, so clean them up.

Annotate deprecated style properties

Use G_PARAM_DEPRECATED with deprecated style properties.
This will make it easier to identify and remove such stale
properties from css, since it will now trigger warnings.

css provider: Issue deprecation warnings for style properties

This makes G_PARAM_DEPRECATED useful for style properties.

cssprovider: Only push a new scanner when none exists

For @import, we want to emit the error as part of the @import statement,
not as part of the new file.

cssprovider: Have a section on import error

When loading a nonexisting CSS file using
gtk_css_provider_load_from_file() or gtk_css_provider_load_from_path()
we would emit the error using a NULL scanner. Don't do that, because
we'll have a NULL section in that case and error handlers don't like
that.

Testcase attached.

https://bugzilla.redhat.com/show_bug.cgi?id=1277959

gtkcontainer: clarify requirements for implementations

Spent quite a few time investigating why the widgets were
not shown in my GtkContainer implementation.

https://bugzilla.gnome.org/show_bug.cgi?id=758087

Updated Scottish Gaelic translation

spinbutton: Document some style properties as deprecated

No code change.

range: Document some style properties as deprecated

Remove dead code dealing with the arrow-displacement-x/y style
properties, and document them as deprecated.

progressbar: Ignore x/yspacing style properties

These are not really useful, so ignore and deprecate them.

menuitem: Document some style properties as deprecated

No code change.

menubar: Ignore the shadow-type style property

The border can be controlled with regular CSS, so deprecate
and ignore this style property.

icon view: Document some style properties as deprecated

No code change.

entry: Ignore the icon-prelight style property

The icon prelight effect can be controlled with regular CSS,
so deprecate and ignore this style property.

button: Document some style properties as deprecated

No code change.

button box: Document some style properties as deprecated

No code change.

assistant: Document style properties as deprecated

No code change.

Avoid an unused variable warning

This was introduced with the recent win32 build fix.

stack switcher: fix typos in documentation comment

Updated Norwegian bokmål translation.

label: Use css nodes for links

Use a subnode with name link for links in labels. These subnodes
carry the :link or :visited state.

x11: Do not misdetect mice with abs axes as touchscreens

VMWare seems to create mouse devices with abs axes which confuses
our detection of single-touch touchscreens. Those have though a
name we can match on ("VirtualPS/2 VMware VMMouse"), it should
be pretty safe to assume that no real touchscreens have "mouse"
in their name...

https://bugzilla.gnome.org/show_bug.cgi?id=757358

wayland: Translate virtual modifiers too

Wayland allows us to receive virtual modifiers too so we can just use
them directly if the compositor does send them.

https://bugzilla.gnome.org/show_bug.cgi?id=748904

spinbutton: update node state more often

We need to update the entry_node state, and we should
update the node state initially.

shortcuts: Support direction-specific shortcuts

The prime example for direction-dependent shortcuts is using
<Alt>Left or <Alt>Right to go back. Support this by adding a
direction property to GtkShortcutsShortcut, and filtering by
the current text direction.

https://bugzilla.gnome.org/show_bug.cgi?id=757888

help overlay: Allow key sequences

Extend the syntax to allow sequences of keys or key combinations,
e.g. t+t or <ctl>c+<ctl>x.

https://bugzilla.gnome.org/show_bug.cgi?id=758051

Add one more selector to CSS parsing tests

Add an E + F selector to the selector parsing tests.
It is a CSS selector that we support.

Always pass the right state in style context getters

Otherwise, we are causing warnings that break make check.

testsuite: Skip GtkFileChooserNative for notification tests

Like all the other GtkFileChooser implementations, this one
emits some unneeded notifications.

HighContrast: Fix selection styling in icon views

This makes selected items in icon view show up again.

HighContrast: Fix selection styling in treeviews

This makes selected rows in treeviews show up again.

HighContrast: Fix selection styling in some places

Make selected flow box children and list box rows show up again.

Adwaita: Misc cleanups

Various cleanups and simplifications.

HighContrast: Fix shadow size

Ensure that the shadow extents don't change as we go to backdrop,
to prevent windows from jumping.

Adwaita: Fix selection styling in icon views

This was also broken by recent selection styling changes.

Adwaita: Fix selection styling in flow boxes

This was broken by recent selection styling changes.

GtkApplicationWindow: the help_overlay is nullable

GtkApplicationWindow: Fix typos

Adwaita: Fix window shadow size

Ensure that the shadow extents don't change as we go to backdrop,
to prevent windows from jumping.

Cosmetic: Avoid explicit state variables

Using the state of the context makes this more obviously correct.

Cosmetic: Avoid explicit state variables

Using the state of the context makes this more obviously correct.

Cosmetic: Avoid explicit state variables

Using the state of the context makes this more obviously correct.

Cosmetic: Avoid explicit state variables

Using the state of the context makes this more obviously correct.

Cosmetic: Avoid explicit state variables

Using the state of the context makes this more obviously correct.

Cosmetic: Avoid explicit state variables

Using the state of the context makes this more obviously correct.

Cosmetic: Avoid explicit state variables

Fold it into its single user.

Cosmetic: Avoid explicit state variables

Using the state of the context makes this more obviously correct.

Cosmetic: Avoid explicit state variables

Using the state of the context makes this more obviously correct.

Cosmetic: Avoid explict state variables

Using the state of the context makes this more obviously correct.

Cosmetic: Avoid an explicit state variable

Using the state of the context makes it more obviously correct.

listbox: Cosmetic cleanup

We don't cast these arguments anywhere else, so lets not do here
either.

window: Don't invalidate cssnode during get_preferred_width()

Getting the shadow width must not call gtk_style_context_set_state()
because that will invalidate the node and cause a style-updated emission
which can cause gtk_widget_queue_resize() calls.

And calling queue_resize() from get_preferred_size() essentially means
the size is permanently invalid because you invalidate it while
querying it.

This causes flickering of windows when going from/to backdrop state. To
avoid this we either need to fix the theme to not have different shadow
sizes in those cases or we need to ensure the window doesn't flicker in
the first place.

Us ... instead of …  for zh_CN

Updated Portuguese translation

GtkShortcutsShortcut: Add missing word

GtkGrid: Remove invalid return annotation